Skip to content

(RND-19052) feat: add vessel tags param to CargoMovement #556

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Ayden666 wants to merge 3 commits into
base: master
Choose a base branch
from
Open

(RND-19052) feat: add vessel tags param to CargoMovement #556

Ayden666 wants to merge 3 commits into from

Conversation

@Ayden666
Copy link
Contributor

@Ayden666 Ayden666 commented Feb 10, 2026 •
edited by atlassian bot
Loading

RELATED TICKETS

! Do not post related PRs here, unless they are open source !

CHANGELOG

  • Add filter_vessel_tags param to CargoMovements which corresponds to vessel_tags param of POST https://api.vortexa.com/v5/cargo/search (link to endpoint doc)
  • Add exclude_vessel_tags param to CargoMovements which corresponds to vessel_tags_excluded param of POST https://api.vortexa.com/v5/cargo/search (link to endpoint doc)

TESTS

  • Tested locally

Copilot AI review requested due to automatic review settings February 10, 2026 17:12
Copilot AI reviewed Feb 10, 2026
View reviewed changes
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for filtering/excluding cargo movements by vessel tags by exposing new parameters on the CargoMovements.search endpoint and mapping them to the API request body.

Changes:

  • Added filter_vessel_tags parameter and mapped it to request body vessel_tags
  • Added exclude_vessel_tags parameter and mapped it to request body vessel_tags_excluded
  • Imported Tag type for these new parameters

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Ayden666